-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update pr template #2013
Update pr template #2013
Conversation
Your Render PR Server URL is https://chainsafe-components-stage-pr-2013.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-c8l6sif6d9kqnpondtag. |
Your Render PR Server URL is https://storage-ui-stage-pr-2013.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-c8l6siv6d9kqnpondtf0. |
Your Render PR Server URL is https://files-ui-stage-pr-2013.onrender.com. Follow its progress at https://dashboard.render.com/static/srv-c8l6sjf6d9kqnpondtsg. |
I'm not sure how I upset the code analysis with such a small change, I followed the same indentation 😅. I'll look again tomorrow. |
This reverts commit f7d2554.
Codacy seems to be failing with two (incorrect?) warnings on the markdown file. I have experimented with their suggested way to ignore files and added a codacy.yml config at the root to exclude the Does anyone have any suggestions to suppress these warnings? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess it's drunk, let's re-run it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now :)
edit: erm no
I deactivated those warnings on codacy directly, they are 100% not helpful and wrong IMO |
@Tbaut Thanks for taking care of this |
Added a new check to prompt us about billing changes being applied to both Files and Storage.